Skip to content

Conversation

GregoryComer
Copy link
Member

@GregoryComer GregoryComer commented Aug 31, 2025

Set up wheel build jobs for Windows (x86-64). This is derived from the torchvision build setup (see https://github.com/pytorch/vision/blob/b208f7f4125e8a9a51638549ccf3e9b163f53dda/.github/workflows/build-wheels-windows.yml#L4).

GregoryComer and others added 30 commits August 26, 2025 15:48
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 10, 2025
ghstack-source-id: 094157c
ghstack-comment-id: 3240461597
Pull-Request: #13837
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 10, 2025
ghstack-source-id: f9f58b1
ghstack-comment-id: 3240461597
Pull-Request: #13837
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 10, 2025
ghstack-source-id: dae6e73
ghstack-comment-id: 3240461597
Pull-Request: #13837
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 10, 2025
ghstack-source-id: 4147e5a
ghstack-comment-id: 3240461597
Pull-Request: #13837
@GregoryComer GregoryComer marked this pull request as ready for review September 10, 2025 04:03
@GregoryComer
Copy link
Member Author

Finally have green CI. I'm rebasing and re-running CI, but it should be ready for review.

@@ -0,0 +1,75 @@
#!/usr/bin/env python
Copy link
Member Author

@GregoryComer GregoryComer Sep 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test logic is intentionally different from the Linux/Mac logic. This is due to needing a bunch of additional environment setup in the job for native build. This Windows test logic is actually a bit more comprehensive, as it uses pybindings and verifies model outputs (vs just running the exported model on the native runner and ensuring it doesn't crash). I'd like to come back and update the other logic post-GA.

# Otherwise, flatc will target the project's toolchain (i.e. iOS, or Android).
ExternalProject_Add(
flatbuffers_ep
PREFIX ${CMAKE_CURRENT_BINARY_DIR}/flatc_proj
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More path length workarounds.

Copy link
Contributor

@swolchok swolchok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a quick comment, don't know much about windows setup here

Copy link
Contributor

@SS-JIA SS-JIA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Btw was a wheel build job triggered to test these changes?

@GregoryComer
Copy link
Member Author

GregoryComer commented Sep 11, 2025

LGTM. Btw was a wheel build job triggered to test these changes?

Yeah, thanks. See the Build Windows Wheels jobs. I'll double-check the resultant wheel one more time locally, but it should be good to go.

@GregoryComer GregoryComer removed the release notes: none Do not include this in the release notes label Sep 11, 2025
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 11, 2025
ghstack-source-id: 9a482cc
ghstack-comment-id: 3240461597
Pull-Request: #13837
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Sep 11, 2025
ghstack-source-id: 8e951e6
ghstack-comment-id: 3240461597
Pull-Request: #13837
@GregoryComer GregoryComer merged commit 56d5186 into main Sep 11, 2025
299 of 308 checks passed
@GregoryComer GregoryComer deleted the gh/GregoryComer/151/head branch September 11, 2025 02:25
@GregoryComer GregoryComer mentioned this pull request Sep 11, 2025
2 tasks
StrycekSimon pushed a commit to nxp-upstream/executorch that referenced this pull request Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries/all Release PRs with this label will build wheels for all python versions ciflow/binaries CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants